Indicate vetting progress to users#291
Conversation
naglepuff
force-pushed
the
issue-279-vetting-progress-ui
branch
from
45a5558 to
6f5fc43
Compare
BryonLewis
left a comment
BryonLewis
left a comment
There was a problem hiding this comment.
The code is perfect for approval, I have more questions about the workflow and some details.
I.E should the submitted recordings be their own recordings or should it be under their shared recordings. Those are really public recordings that have been shared with them.
Maybe the title of that section should be "Shared with Me".
So maybe show submitted recordings should be only tied to the shared with me recordings. Additionally if you have 'allow uploads by non-admin users' I would have the upload and the My Recordings section not displayed in the interface.
|
@BryonLewis we talked a bit about the workflow and design at the meeting today. I'm doing filtering for submitted/unsubmitted recordings for both the user's recordings and shared recordings. I'm thinking about this as a feature that can apply to both. I also fixed the bug we discussed. |
This reverts commit f703be0.
2 participants
About this PR
This PR is another in a series of changes that sets up a community vetting workflow for the BatAI web application. It depends on #281.
Changes
Fix #279
The following acceptance criteria from the linked issue are addressed with this set of changes:
1. A reviewer can filter the list to show only uncompleted files.
This can be done on the main Recordings view:
Or the side bar:
2. Completion state is visible without opening each file.
This is part of #281. A new column was added to the main Recording view that indicates which files have been reviewed and have a submitted annotation for that user.
The following are additional optional changes implemented by this PR:
1. Display a progress summary (for example: 127 / 300 completed).
Todo: